Skip to content

Jacobhylen/flashing upython firmware guide #654

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 16 commits into from
Nov 15, 2022
Merged

Jacobhylen/flashing upython firmware guide #654

merged 16 commits into from
Nov 15, 2022

Conversation

@jacobhylen
Copy link
Contributor

@jacobhylen jacobhylen commented Nov 11, 2022 •
edited
Loading

What This PR Changes

  • Adds missing documentation on how to flash micropython firmware to the boards that can do it.

Contribution Guidelines

@gatsby-cloud
Copy link

gatsby-cloud bot commented Nov 11, 2022 •
edited
Loading

✅ docs-content deploy preview ready

@AileenBanshee AileenBanshee marked this pull request as ready for review November 11, 2022 18:24
AileenBanshee
AileenBanshee reviewed Nov 11, 2022
View reviewed changes
Copy link
Contributor

@AileenBanshee AileenBanshee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wrote you few comments

sebromero
sebromero reviewed Nov 11, 2022
View reviewed changes
Copy link
Contributor

@sebromero sebromero left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added some suggestions.

@sebromero
Copy link
Contributor

sebromero commented Nov 12, 2022

@jacobhylen I tweaked a few things. For the Portenta H7 you cannot use bossac but you need to use dfu util. I adjusted it. I didn't know the path on windows though and I kept it very brief since I was on the train. We should also add the links for Linux. Can you please have a look? Thank you!

karlsoderby
karlsoderby requested changes Nov 15, 2022
View reviewed changes
Copy link
Contributor

@karlsoderby karlsoderby left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @jacobhylen, left a series of suggestions and a few comments! Please check it out :)

jacobhylen and others added 3 commits November 15, 2022 10:06
@jacobhylen @karlsoderby
Update content/tutorials/generic/micropython-installation/micropython…
7cbfb1b 
…-installation.md

Co-authored-by: Karl Söderby <35461661+karlsoderby@users.noreply.github.com>
@jacobhylen @karlsoderby
Apply suggestions from code review
0003cd1 
Co-authored-by: Karl Söderby <35461661+karlsoderby@users.noreply.github.com>
@jacobhylen
Update micropython-installation.md
40c8203
@karlsoderby karlsoderby merged commit 65ec67a into main Nov 15, 2022
@karlsoderby karlsoderby deleted the jacobhylen/flashing-upython-firmware-guide branch November 15, 2022 11:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sebromero sebromero sebromero left review comments

@karlsoderby karlsoderby karlsoderby approved these changes

@ubidefeo ubidefeo Awaiting requested review from ubidefeo

@BenjaminDannegard BenjaminDannegard Awaiting requested review from BenjaminDannegard

+1 more reviewer

@AileenBanshee AileenBanshee AileenBanshee left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@jacobhylen @sebromero @karlsoderby @AileenBanshee